Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add documentation about Xstate and configuration handling #87

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add documentation about Xstate and configuration handling #87

wants to merge 2 commits into from
+145 −12

Conversation

spaceo
Copy link
Contributor

@spaceo spaceo commented Jan 13, 2025 •
edited
Loading

Link to issue

https://reload.atlassian.net/browse/DDFBRA-249

Description

Added Architectural Decision Rocords describing the choices around configuration handling and the Xstate state machine.

@spaceo spaceo force-pushed the DDFBRA-249-tilfoj-dokumentation-pa-xstate-og-config-systemet branch from 22101e4 to b5a7e5b Compare January 13, 2025 15:40
@spaceo
Add documentation about configuration handling
b97fedf 
...and move documentation into `/docs` since that is where it needs to
reside in order to be fetched by our documentation site.
@spaceo spaceo force-pushed the DDFBRA-249-tilfoj-dokumentation-pa-xstate-og-config-systemet branch 2 times, most recently from 2bedc38 to 7f0ecab Compare January 14, 2025 19:35
@spaceo spaceo force-pushed the DDFBRA-249-tilfoj-dokumentation-pa-xstate-og-config-systemet branch from 7f0ecab to 9498ccb Compare January 14, 2025 19:39
@spaceo spaceo force-pushed the DDFBRA-249-tilfoj-dokumentation-pa-xstate-og-config-systemet branch from 9498ccb to e0f01f8 Compare January 15, 2025 11:58
@spaceo spaceo marked this pull request as ready for review January 15, 2025 11:59
ThomasGross
ThomasGross approved these changes Jan 15, 2025
View reviewed changes
Copy link
Contributor

@ThomasGross ThomasGross left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ThomasGross ThomasGross ThomasGross approved these changes

Assignees

@Dresse Dresse

@ThomasGross ThomasGross

@Adamik10 Adamik10

@kasperbirch1 kasperbirch1

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@spaceo @ThomasGross @Dresse @Adamik10 @kasperbirch1